Skip to content
This repository has been archived by the owner on May 26, 2022. It is now read-only.

test: libp2p transport test suite #93

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

marten-seemann
Copy link
Collaborator

Taking over #13. Can't figure out how to commit to the original branch and continue that PR.

@marten-seemann
Copy link
Collaborator Author

marten-seemann commented Dec 7, 2019

The tests are failing because we're running into the StressTestTimeout (60s). It's just that without a decent test framework, we don't get a helpful error message.

@Stebalien
Copy link
Member

@marten-seemann it would be really nice to get this passing.

@liamsi
Copy link

liamsi commented Jul 11, 2021

I've updated to the latest test-suite and made the build pass again: https://github.com/liamsi/go-libp2p-quic-transport/tree/transport-test-updated

The tests fail locally but with other fails than StressTestTimeout (60s) (or at least I don't see it if this happens under the hood): https://github.com/liamsi/go-libp2p-quic-transport/runs/3039813604

Edit: Although it seems the SubtestStress1Conn1Stream100Msg is the only one that panics.

@Stebalien Stebalien force-pushed the transport-tests branch 2 times, most recently from 8b5b2b1 to 88af285 Compare July 22, 2021 23:49
@Stebalien
Copy link
Member

@liamsi pulled.

@Stebalien Stebalien force-pushed the transport-tests branch 3 times, most recently from 70c0573 to 64ec128 Compare July 23, 2021 01:42
@Stebalien
Copy link
Member

Ok, after libp2p/go-libp2p-testing#34, everything but windows is passing, and windows is failing consistently. This looks like it might be a QUIC bug.

@Stebalien
Copy link
Member

Hm. I take that back? I'm getting:

  stream_suite.go:241: timeout: no recent network activity

On ubuntu as well, but not as reliably.

Also, after a quite a few runs, I'm getting "open sockets", even though I'm cleaning them up. Clearly we're something is failing to garbage collect (could be the test?).

@Stebalien
Copy link
Member

Also, after a quite a few runs, I'm getting "open sockets", even though I'm cleaning them up. Clearly we're something is failing to garbage collect (could be the test?).

Ok, that's just us hanging on to old sessions.

@Stebalien
Copy link
Member

"retired connections"

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants